Skip to content

Add Simple Expression constraint#2980

Merged
lesaltzm merged 11 commits into
mainfrom
lesaltzm/simplexpr
Oct 7, 2025
Merged

Add Simple Expression constraint#2980
lesaltzm merged 11 commits into
mainfrom
lesaltzm/simplexpr

Conversation

@lesaltzm
Copy link
Copy Markdown
Contributor

@lesaltzm lesaltzm commented Oct 3, 2025

Some properties that depend on ThisItem scope will have significant performance costs to async operations. We've discussed and landed on restricting the set of things that can be accessed from those properties to just sync functions, the thisitem scope, enums, and operators.

This PR is a first pass at this, but it won't be immediately exposed to customers, so we'll iterate on it.

@lesaltzm lesaltzm requested a review from a team as a code owner October 3, 2025 20:44
Comment thread src/libraries/Microsoft.PowerFx.Core/Texl/Builtins/If.cs
@CarlosFigueiraMSFT
Copy link
Copy Markdown
Contributor

        return new BindingConfig(

Need to pass the enforceSimpleExpressions parameter to the .ctor when cloning.


Refers to: src/libraries/Microsoft.PowerFx.Core/Binding/BindingConfig.cs:57 in 8f2a273. [](commit_id = 8f2a273, deletion_comment = False)

Comment thread src/strings/PowerFxResources.en-US.resx
@jas-valgotar
Copy link
Copy Markdown
Contributor

✅ No public API change.

@lesaltzm lesaltzm enabled auto-merge (squash) October 7, 2025 16:39
Copy link
Copy Markdown
Contributor

@CarlosFigueiraMSFT CarlosFigueiraMSFT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@jas-valgotar
Copy link
Copy Markdown
Contributor

✅ No public API change.

@lesaltzm lesaltzm merged commit 0330bc1 into main Oct 7, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants